The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	May 7, 2026 7:59am

PR Summary

Medium Risk
Touches workflow migration, import parsing, and persistence writeback logic; mistakes could drop or mis-type subBlock values during load/import, but changes are scoped and covered by new tests.

Overview
Improves legacy workflow compatibility by repairing malformed subBlocks (missing/invalid id/type, type: "unknown", non-object entries, and the literal "undefined" key) while preserving values when the subBlock can be mapped to a known field.

Extends migrateSubblockIds to sanitize subBlocks in addition to renaming legacy IDs (e.g., knowledgeBaseId → knowledgeBaseSelector), and updates import parsing to unwrap API data envelopes, extract workflow names from workflow.name, and run the same subBlock migration/sanitization on imported JSON.

Updates normalized-table migration persistence to track per-block updatedAt and use it in the UPDATE predicate to avoid overwriting newer DB changes when writing back migrated blocks.

^{Reviewed by Cursor Bugbot for commit 0942555. Configure here.}

Greptile Summary

This PR introduces a sanitizeMalformedSubBlocks utility that repairs legacy subBlock state — dropping literal-key "undefined" entries, subblocks with type: "unknown" that have no registry config, and entries missing required id/type fields, while preserving values wherever possible. It also adds API-envelope unwrapping to the import path so workflows exported in the { data: { state: ... } } format round-trip correctly.

• New subblocks.ts sanitizer — walks every subBlock entry, looks up the block config for the correct type, repairs or drops malformed metadata, and returns a changed flag so the migration pipeline only writes back when something actually changed.
• Pipeline reordering in utils.ts — subblock-ID migration now runs before credential migration, and a null-guard prevents a crash when a non-object subBlock is encountered in migrateCredentialIds.
• Optimistic-lock writes in persistMigratedBlocks — the write now includes workflowId scope and an updatedAt equality check, preventing stale migrations from overwriting concurrent edits.

Confidence Score: 4/5

Safe to merge; the sanitization logic is well-tested and the migration pipeline is additive with no destructive data changes.

The core repair logic is covered by targeted tests and the changes are backwards-compatible. The main rough edges are a redundant second sanitization pass in the import path, a missing log for sanitize-only writes, and concurrent migration misses going unrecorded — none of which affect correctness under normal load.

packages/workflow-persistence/src/load.ts (silent concurrent-update miss) and apps/sim/lib/workflows/operations/import-export.ts (double sanitization pass).

Important Files Changed

_{Reviews (1): Last reviewed commit: "fix(sanitize): repair malformed subblock..." | Re-trigger Greptile}